Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Npm: Do not fail hard if enriching incomplete data from the NPM registry fails #5675

Merged
merged 2 commits into from
Aug 23, 2022

Conversation

sschuberth
Copy link
Member

This allows to handle packages that were never published to the NPM
registry but only ever referenced by (short) repository URL.

Fixes #5632.

…try fails

This allows to handle packages that were never published to the NPM
registry but only ever referenced by (short) repository URL.

Fixes #5632.

Signed-off-by: Sebastian Schuberth <sschuberth@gmail.com>
This is the same code as in the base class.

Signed-off-by: Sebastian Schuberth <sschuberth@gmail.com>
@sschuberth sschuberth requested a review from a team as a code owner August 23, 2022 09:06
@codecov
Copy link

codecov bot commented Aug 23, 2022

Codecov Report

Merging #5675 (675477e) into main (1403877) will decrease coverage by 0.02%.
The diff coverage is 84.61%.

@@             Coverage Diff              @@
##               main    #5675      +/-   ##
============================================
- Coverage     65.50%   65.47%   -0.03%     
- Complexity     2212     2219       +7     
============================================
  Files           271      271              
  Lines         16580    16592      +12     
  Branches       3432     3445      +13     
============================================
+ Hits          10860    10863       +3     
- Misses         4577     4586       +9     
  Partials       1143     1143              
Flag Coverage Δ
funTest-analyzer-docker 74.49% <84.61%> (+0.01%) ⬆️
funTest-non-analyzer 46.13% <ø> (-0.01%) ⬇️
test 32.06% <38.46%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
analyzer/src/main/kotlin/managers/Pnpm.kt 95.45% <ø> (-0.38%) ⬇️
analyzer/src/main/kotlin/managers/Npm.kt 52.04% <84.61%> (-0.44%) ⬇️
...li/src/main/kotlin/commands/RequirementsCommand.kt 1.08% <0.00%> (-0.02%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@sschuberth sschuberth merged commit 7aadaf8 into main Aug 23, 2022
@sschuberth sschuberth deleted the npm-pkg-no-registry branch August 23, 2022 11:58
@sschuberth sschuberth added the release notes Changes that should be mentioned in release notes label Aug 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release notes Changes that should be mentioned in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NPM dependencies that point to Git / GitHub repos cannot be processed anymore
2 participants